Switch to using a systemd generator for /var
authorColin Walters <walters@verbum.org>
Thu, 11 May 2017 18:54:12 +0000 (14:54 -0400)
committerAtomic Bot <atomic-devel@projectatomic.io>
Tue, 16 May 2017 16:13:05 +0000 (16:13 +0000)
commit30705889cb867c18cdb7fed8e55dc46477c069fa
treeb6247fa4e4d51ad7a44bfef90381f3aad47c9d52
parentd815ba2a81ad14d9d4edc31dcd282dcc2a3a8fb9
Switch to using a systemd generator for /var

If one wants to set up a mount for `/var` in `/etc/fstab`, it
won't be mounted since `ostree-prepare-root` set up a bind mount for
`/var` to `/sysroot/ostree/$stateroot/var`, and systemd will take
the already extant mount over what's in `/etc/fstab`.

There are a few options to fix this, but what I settled on is parsing
`/etc/fstab` in a generator (exactly like `systemd-fstab-generator` does),
except here we look for an explicit mount for `/var`, and if one *isn't* found,
synthesize the default ostree mount to the stateroot. Another nice property is
that if an admin creates a `var.mount` unit in `/etc` for example, that will
also override our mount.

Note that today ostree doesn't hard depend on systemd, so this behavior only
kicks in if we're built with systemd *and* libmount support (for parsing
`/etc/fstab`).  I didn't really test that case though.

Initially I started writing this as a "pure libc" program, but at one point
decided to use `libostree.so` to find the booted deployment. That didn't work
out because `/boot` wasn't necessarily mounted and hence we couldn't find the
bootloader config. A leftover artifact from this is that the generator code
calls into libostree via the "cmd private" infrastructure. But it's an easy way
to share code, and doesn't hurt.

Closes: #859
Approved by: jlebon
Makefile-libostree.am
Makefile-switchroot.am
configure.ac
src/libostree/ostree-cmdprivate.c
src/libostree/ostree-cmdprivate.h
src/libostree/ostree-impl-system-generator.c [new file with mode: 0644]
src/libostree/ostree-sysroot-deploy.c
src/switchroot/ostree-mount-util.h
src/switchroot/ostree-prepare-root.c
src/switchroot/ostree-system-generator.c [new file with mode: 0644]